Skip to content

Add optional verbose configuration to .bazelrc#737

Merged
mhucka merged 5 commits intomasterfrom
mh-add-bazelrc-verbose
Jun 10, 2025
Merged

Add optional verbose configuration to .bazelrc#737
mhucka merged 5 commits intomasterfrom
mh-add-bazelrc-verbose

Conversation

@mhucka
Copy link
Copy Markdown
Collaborator

@mhucka mhucka commented May 29, 2025

This adds a config that can be enabled using the option --config=verbose. It it is off by default, keeping the previous behavior intact. When enabled, it makes build and test runs produce a modest increase in diagnostic output. This is useful when debugging GitHub Actions workflows.

Also in this PR: some comments acting as section dividers to make navigating this file easier. (I find it helpful when trying to figure out where a given change should go.)

Finally, this PR documents the --config=verbose flag in docs/bazel.md.

mhucka added 2 commits May 29, 2025 23:03
This adds a config that can be enabled using the option
`--config=verbose`. It it is off by default, keeping the previous
behavior intact. When enabled, it makes build and test runs produce a
modest increase in diagnostic output. This is useful when debugging
GitHub Actions workflows.

Also in this commit: some comments acting as section dividers to make
navigating this file easier. (I find it helpful when trying to figure
out where a given change should go.)
@mhucka mhucka changed the title Add verbose configuration flags Add verbose configuration option to .bazelrc May 29, 2025
@mhucka mhucka changed the title Add verbose configuration option to .bazelrc Add optional verbose configuration to .bazelrc May 30, 2025
@mhucka mhucka self-assigned this May 30, 2025
@mhucka mhucka requested review from dstrain115 and pavoljuhas May 30, 2025 18:12
Copy link
Copy Markdown
Collaborator

@pavoljuhas pavoljuhas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this is a permanent bazel option it should be documented so others can discover it without reading bazelrc. If needed temporarily to fix bazel builds, consider adding a TODO to clean it up later.

@mhucka
Copy link
Copy Markdown
Collaborator Author

mhucka commented Jun 10, 2025

If this is a permanent bazel option it should be documented so others can discover it without reading bazelrc. If needed temporarily to fix bazel builds, consider adding a TODO to clean it up later.

Good point. I added an explanation to docs/bazel.md.

@mhucka mhucka merged commit e5e335e into master Jun 10, 2025
15 of 19 checks passed
@mhucka mhucka deleted the mh-add-bazelrc-verbose branch June 10, 2025 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants